Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding SPLAC beside PLAC #520

Draft
wants to merge 10 commits into
base: v7.1
Choose a base branch
from
Draft

Adding SPLAC beside PLAC #520

wants to merge 10 commits into from

Conversation

tychonievich
Copy link
Collaborator

Conversation draft of adding place records to 7.1

This puts PLAC and SPLAC side-by-side, like NOTE and SNOTE. That is not the only possibility, and alternative PRs with other designs are anticipated to allow better comparison of the options.

This only addresses the record-vs-substucture topic. The additional substuctures that have also been considered for location records (for example, see GEDCOM-L's _LOC extension) would presumably be added to the new <<PLACE_DETIALS>> production in a future PR if we decide that this organization is the right approach.

This puts PLAC and SPLAC side-by-side, like NOTE and SNOTE. That is not the only possibility, and alternative PRs with other designs are anticipated to allow better comparison of the options.

This only addresses the record-vs-substucture topic. The additional substuctures that have also been considered for PLAC/_LOC/etc are prepared for by creating a `<<PLACE_DETIALS>>` production to be the home for such additions in a future PR.
specification/gedcom-3-structures-1-organization.md Outdated Show resolved Hide resolved

The `<<SHARED_PLACE_STRUCTURE>>` inside a `SHARED_PLACE_RECORD` points to larger jurisdictions that this place is a part of.
If a city is part of a county which is part of a state, the city's place record should point to the county's place record, not the states.
Multiple `<<SHARED_PLACE_STRUCTURE>>`s are permitted to support places within multiple hierarchies (for example, a church that's both within an ecclesiastical region and a political region).
Copy link
Collaborator

@dthaler dthaler Aug 6, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we need a TYPE/KIND to distinguish ecclesiastical, political, etc. containment. The _LOC extension does this with HIERARCHICAL_RELATIONSHIP.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The HIERARCHICAL_RELATIONSHIP is needed (per existing _LOC usage) inside a "shared place structure" within a SHARED_PLACE_RECORD, but not from within a PLACE_REFERENCE since the latter is not a containment relationship. This suggests that using <<SHARED_PLACE_STRUCTURE>> in both places may not be appropriate.

specification/gedcom-3-structures-1-organization.md Outdated Show resolved Hide resolved
specification/gedcom-3-structures-1-organization.md Outdated Show resolved Hide resolved
specification/gedcom-3-structures-1-organization.md Outdated Show resolved Hide resolved
specification/gedcom-3-structures-1-organization.md Outdated Show resolved Hide resolved
specification/gedcom-3-structures-1-organization.md Outdated Show resolved Hide resolved
A `voidPtr` and `PHRASE` can be used to describe places not referenced by any `SPLAC` record, but so can a `PLAC` structure. Using a `voidPtr` with `SPLAC` is not recommended.

:::example
The following both indicate that a birth happened "at home" with no additional details on where that was. The second version is preferred; the first should not be used.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm less certain that the second version is preferred, since HEAD.PLAC.FORM would apply in the second version, where "at home" would then be labeled a "city" in UI. So we should discuss more.

I'd also ask the question: what is the meaning of HEAD.PLAC.FORM if only SPLAC is used everywhere? If none, then perhaps we need a sentence saying not to use HEAD.PLAC.FORM in that case.

specification/gedcom-3-structures-3-meaning.md Outdated Show resolved Hide resolved
@dthaler dthaler mentioned this pull request Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants